-
Notifications
You must be signed in to change notification settings - Fork 117
Add swagger support for SOFARPC restful #131
Conversation
Users can add swagger dependencies and add |
Codecov Report
@@ Coverage Diff @@
## master #131 +/- ##
===========================================
+ Coverage 57.33% 58.3% +0.96%
- Complexity 417 436 +19
===========================================
Files 54 56 +2
Lines 1751 1806 +55
Branches 313 318 +5
===========================================
+ Hits 1004 1053 +49
- Misses 541 548 +7
+ Partials 206 205 -1
Continue to review full report at Codecov.
|
import com.alipay.sofa.rpc.boot.config.SofaBootRpcConfigConstants; | ||
import com.alipay.sofa.rpc.boot.config.SofaBootRpcProperties; | ||
import com.alipay.sofa.rpc.boot.config.ZookeeperConfigurator; | ||
import com.alipay.sofa.rpc.boot.config.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should avoid using import * here. You can change idea settings from Editor - Code Style - Java - Imports - import with '*'
to 999
etc and then format code again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed as you suggested.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
we will not change version to stable version number until we prepare to release, so we can release some snapshot for test. |
@leizhiyuan You mean |
ok, I see,sofaboot-dependencies 310 has released four days ago. |
after ci,I will merge this pr to master |
Add swagger support for SOFARPC restful services.